Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add NativeAnnotationAttribute to the metadata file #2001

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dpaoliello
Copy link
Contributor

@dpaoliello dpaoliello commented Sep 23, 2024

ClangSharp has recently started emitting the NativeAnnotation attribute for C++ annotate attributes: dotnet/ClangSharp#552

This PR adds the definition of NativeAnnotationAttribute to metadata.cs.

@mikebattista
Copy link
Collaborator

Is this wired up end-to-end? Are we using a version of clangsharp that emits this? Were there any changes to the baseline?

@dpaoliello
Copy link
Contributor Author

I stumbled on this when I tried to update the version of ClangSharp we were using internally - not sure if this is being used end-to-end or is in the baseline (I honestly don't know how to check).

@riverar
Copy link
Collaborator

riverar commented Dec 3, 2024

Just ran into this myself now. Had clangsharppinvokegenerator 18.1.0.2 installed. (Restoring to metadata-supported 17.0.1 resolved the issue.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants